-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add more command: continue, step, ... #62
Conversation
Speaking of which, I can't find |
Just to add context, I'm working on implementing command that are used in V8 to support Wasm debugging. See e.g. https://github.com/kishiguro/chromium_v8/blob/5d8f3b46a570b25e27c492e86455072012601f90/src/debug/wasm/gdb-server/target.cc (and my implementation at https://github.com/yurydelendik/rust-gdb-remote-protocol/tree/wasm)
I can place all deprecated commands under a feature flag
They perhaps coming from LLDB extension, e.g. https://opensource.apple.com/source/lldb/lldb-310.2.36/docs/lldb-gdb-remote.txt.auto.html |
Please do, thank you :) Let me know when you're done, and I'll merge this PR - it looks good!
Ah, that's it, thanks! |
FWIW they are only really deprecated if the remote supporst See #26 for some info on the difficulties of |
Not really sure how to use cfg() with nom macros -- added separate parser-functions |
@jD91mZM2 I think it is all set for this PR. |
Sorry for the delay! |
Add support for:
qProcessInfo
qRegisterInfo
qSymbol
c
-- continues
-- stepExtends support for 'H' -- supports different targets.